Skip to content

[feat][backend]: response API context refill#415

Merged
Ev4nFeng merged 25 commits into
mainfrom
feat/auto_task_resp_id
Feb 4, 2026
Merged

[feat][backend]: response API context refill#415
Ev4nFeng merged 25 commits into
mainfrom
feat/auto_task_resp_id

Conversation

@Ev4nFeng
Copy link
Copy Markdown
Collaborator

@Ev4nFeng Ev4nFeng commented Jan 30, 2026

What type of PR is this?

Check the PR title

  • This PR title match the format: [<type>][<scope>] <description>. For example: [fix][backend] flaky fix
  • The description of this PR title is user-oriented and clear enough for others to understand.
  • Add documentation if the current PR requires user awareness at the usage level.
  • This PR is written in English. PRs not in English will not be reviewed.

(Optional) Translate the PR title into Chinese

(Optional) More detailed description for this PR(en: English/zh: Chinese)

en:
zh(optional):

(Optional) Which issue(s) this PR fixes

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 30, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 84.51883% with 37 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...bservability/domain/trace/service/trace_service.go 87.50% 11 Missing and 9 partials ⚠️
...bservability/domain/trace/entity/loop_span/span.go 90.00% 4 Missing and 2 partials ⚠️
...ility/domain/trace/service/trace_export_service.go 33.33% 2 Missing and 2 partials ⚠️
...main/task/service/taskexe/tracehub/span_trigger.go 25.00% 2 Missing and 1 partial ⚠️
...domain/task/service/taskexe/tracehub/subscriber.go 40.00% 2 Missing and 1 partial ⚠️
.../domain/task/service/taskexe/tracehub/trace_hub.go 0.00% 1 Missing ⚠️

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #415      +/-   ##
==========================================
+ Coverage   70.85%   71.04%   +0.18%     
==========================================
  Files         624      624              
  Lines       61196    61569     +373     
==========================================
+ Hits        43359    43739     +380     
+ Misses      14690    14658      -32     
- Partials     3147     3172      +25
Flag Coverage Δ
unittests 71.04% <84.51%> (+0.18%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...y/domain/task/service/taskexe/tracehub/backfill.go 67.44% <100.00%> (-0.13%) ⬇️
...s/observability/infra/mq/consumer/task_consumer.go 100.00% <100.00%> (ø)
.../domain/task/service/taskexe/tracehub/trace_hub.go 0.00% <0.00%> (ø)
...main/task/service/taskexe/tracehub/span_trigger.go 80.25% <25.00%> (+4.23%) ⬆️
...domain/task/service/taskexe/tracehub/subscriber.go 72.79% <40.00%> (-1.26%) ⬇️
...ility/domain/trace/service/trace_export_service.go 72.29% <33.33%> (-0.76%) ⬇️
...bservability/domain/trace/entity/loop_span/span.go 79.68% <90.00%> (+3.75%) ⬆️
...bservability/domain/trace/service/trace_service.go 86.45% <87.50%> (+2.63%) ⬆️

... and 12 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9866134...ccd0f6e. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

JqRrt
JqRrt reviewed Feb 4, 2026
View reviewed changes
Comment thread backend/modules/observability/domain/trace/entity/loop_span/span.go
JqRrt
JqRrt reviewed Feb 4, 2026
View reviewed changes
Comment thread backend/modules/observability/domain/trace/entity/loop_span/span.go Outdated
taoyifan89
taoyifan89 reviewed Feb 4, 2026
View reviewed changes
Comment thread backend/modules/observability/application/task.go Outdated
Comment thread backend/modules/observability/domain/task/service/taskexe/tracehub/span_trigger.go Outdated
Comment thread backend/modules/observability/domain/trace/service/trace_service.go
Comment thread backend/modules/observability/domain/trace/service/trace_service.go Outdated
Comment thread backend/modules/observability/pkg/consts/span.go Outdated
@Ev4nFeng Ev4nFeng requested review from JqRrt and taoyifan89 February 4, 2026 07:38
@Ev4nFeng Ev4nFeng requested a review from kidkidkid February 4, 2026 11:16
@Ev4nFeng Ev4nFeng merged commit 1693593 into main Feb 4, 2026
16 checks passed
@Ev4nFeng Ev4nFeng deleted the feat/auto_task_resp_id branch February 4, 2026 11:30
HearyShen pushed a commit that referenced this pull request Feb 26, 2026
@Ev4nFeng @HearyShen
[feat][backend]: response API context refill (#415)
b2c952d 
* feat(backend): response id context refill

* feat(backend): lint & mock

* feat(backend): fix UT

* feat(backend): add some log

* feat(backend): log

* feat(backend): log

* feat(backend): log

* feat(backend): time range

* feat(backend): debug log

* feat(backend): debug log

* feat(backend): bug fix

* feat(backend): bug fix

* feat(backend): some log

* feat(backend): some log

* feat(backend): some log

* feat(backend): add current span

* feat(backend): multi sub fix

* feat(backend): loop span resp id

* feat(backend): raw span resp id

* feat(backend): solve cmts

* feat(backend): remove logs

* feat(backend):

* feat(backend): UT

* feat(backend): log info

* feat(backend): remove handle resp id
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kidkidkid kidkidkid kidkidkid approved these changes

@taoyifan89 taoyifan89 taoyifan89 approved these changes

@JqRrt JqRrt JqRrt approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Ev4nFeng @kidkidkid @taoyifan89 @JqRrt